Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Omit words longer than 80 characters from the search index #1809

Merged
merged 2 commits into from
Jun 22, 2022
Merged

Omit words longer than 80 characters from the search index #1809

merged 2 commits into from
Jun 22, 2022

Conversation

mattheww
Copy link
Contributor

Closes #1735

@mattheww
When creating the search index, omit words longer than 80 characters
d65ce55 
This avoids creating deeply nested objects in searchindex.json
@mattheww
Test that long words are omitted from the search index.
00a55b3 
Note they do appear in the 'docs' part of searchindex.json (so they will be
visible in search teasers).
@Dylan-DPC
Copy link
Member

Thanks

@Dylan-DPC Dylan-DPC merged commit 93aee64 into rust-lang:master Jun 22, 2022
@Dylan-DPC Dylan-DPC mentioned this pull request Jun 22, 2022
Merged
@Dylan-DPC
Copy link
Member

@mattheww can you reöpen this if possible? (or open a new pr with the same branch)

@mattheww
Copy link
Contributor Author

The update to elasticlunr 3.0.0 broke this (it removes add_doc_with_tokenizer). I'll have a look.

@mattheww mattheww mentioned this pull request Jun 26, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

SIGSEGV on long line
2 participants
@mattheww @Dylan-DPC